Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix the wrong objectID in resolveVineyardObject. #2014

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix the wrong objectID in resolveVineyardObject. #2014

wants to merge 1 commit into from

Conversation

dashanji
Copy link
Member

@dashanji dashanji commented Nov 14, 2024 •
edited
Loading

What do these changes do?

As titled

Related issue number

Fixes the downstream case alibaba/GraphScope#4314

import graphscope
import vineyard
from graphscope.framework.loader import Loader

graphscope.set_option(show_log=True)
graphscope.set_option(log_level='debug')

graph = graphscope.g()
graph = graph.add_vertices(Loader('s3://modern_graph/person.csv', key=key, secret=secret, client_kwargs={'region_name': 'us-east-1'}, delimiter='|'), label='vertex')
graph = graph.add_edges(Loader('s3://modern_graph/knows.csv', key=key, secret=secret, client_kwargs={'region_name': 'us-east-1'}, delimiter='|'), label='edge')
print(graph)

session.close()

Get the following error

Loading vertex labeled vertex and:   0%|          | 0/10 [00:00<?, ?it/s]I1113 15:19:44.000000 2867282 /usr/local/include/graphscope/core/loader/arrow_fragment_loader.h:429] read vertex table from vineyard: o64c9191e14c00015
I1113 15:19:44.000000 2867688 /workspaces/v6d/modules/graph/loader/arrow_fragment_loader.cc:151] loading table from vineyard: o04c9191e14c00015, part id = 0, part num = 1
E1113 15:19:44.000000 2867688 /workspaces/v6d/modules/graph/loader/arrow_fragment_loader.cc:432] Failed to read from stream o04c9191e14c00015: Object not exists: failed to get metadata for 'o04c9191e14c00015': failed to read get_data reply: {"content":null,"type":"get_data_reply"}

@dashanji dashanji mentioned this pull request Nov 14, 2024
Open
@github-actions github-actions bot added the stale label Dec 12, 2024
@github-actions GitHub Actions
Copy link
Contributor

/cc @sighingnow, this issus/pr has had no activity for a long time, please help to review the status and assign people to work on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sighingnow sighingnow Awaiting requested review from sighingnow

@siyuan0322 siyuan0322 Awaiting requested review from siyuan0322

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dashanji